CIE: Make the RGB to Lab fast paths work with non-sRGB primaries

The older constants were tied to sRGB primaries. Using the matrices
from the Babl space removes this restriction.

https://bugzilla.gnome.org/show_bug.cgi?id=789695

CIE: Make the RGB to Lab fast paths as accurate as the reference

The matrices used by the fast paths to convert from RGB to XYZ didn't
exactly match the reference matrices in the Babl space. This caused a
measurable error in their output.

https://bugzilla.gnome.org/show_bug.cgi?id=789695

CIE: Use cbrt instead of pow for the reference XYZ to LAB conversion

The fast-paths use an inlining-friendly version of cbrt(3). Using
something similar removes superficial differences between the two
conversion paths. It's not like the C library's cbrt(3) will perform
any worse than its own pow(3).

https://bugzilla.gnome.org/show_bug.cgi?id=789695

CIE: Add conversion from "RGBA float" to "CIE Lab float"

Conversions from "RaGaBaA float" to "CIE Lab float", as seen when
using gegl:shadows-highlights" go via:
  "RaGaBaA float" to "RGBA float"
  "RGBA float"    to "RGB float"
  "RGB float"     to "CIE Lab float"

A direct conversion from "RaGaBaA float" to "CIE Lab float" in simple
C is hindered by the need to check every pixel's alpha value to avoid
dividing by zero. The pipeline stalls make it lose out to the look-up
table and SIMD based conversions to unassociated alpha.

However, we can trivially cut out the second step and still reduce
some memory traffic.

https://bugzilla.gnome.org/show_bug.cgi?id=789695

babl: use snprintf instead of sprintf

Using sprintf with environment variables is dangerous, because it can
easily lead to out of boundary writes on heap space.

While at it, replace sprintf calls with snprintf where proper
boundary checks are possible and required.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

babl: properly handle large files on 32 bit systems

If large file support is enabled on 32 bit systems, it is possible
to trigger an out of boundary write with files larger than 2 GB.

Always check if fseek and ftell are successful and if the file is
small enough to fit into memory.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

babl: always end strncpy strings with NUL

The function strncpy(3) does not guarantee to end the destination string
with NUL character if not enough space was available. This could happen
on systems which allow paths which are longer than 4096 characters.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>

post release version bump

release 0.1.34, update NEWS

export-symbols: add babl_backtrack for gimp-2.8

babl: rearrange BablModel struct

Babl is doing casting that relies on BablFormat / BablModel having the same
layout for shared members.

babl: do not use model.data for storing original model backpointer

Fixing bug #788471

update NEWS

post-release version bump

release 0.1.32

docs: update with api changes

babl: acknowledge some used trademarks

babl: move to/from xyz and rgbtoxyz matrix functions out of babl.h

Both of these are useful, but we might want a wider range, permitting
both linear and non-linear variants to be converted to both XYZ and LAB.

The matrix is for now used in babl for the RGB->Y conversion - this might
warrant a more direct API.

babl: rename babl_space_from_icc to babl_icc_make_space

Thus being more consistent that functions that take icc_data + length have the
prefix babl_icc_, since each icc space in the future potentially can have more
spaces than the relative-colorimetric space defined, _make_ seems better than
_to_ or _new_.

babl: fix clang warnings in babl-polynomial

Cast the input polynomial from BablPolynomial* to
BablBigPolynomial* in the evaluator functions, so that clang
doesn't warn us about out-of-bounds access to its coefficient
array.

HSL: make hue be set in all switch cases for rgb_to_hsl

babl: revert and clarify libtool flag for win32 builds

babl: improve alpha calc. in pal+alpha -> rgba conversion

... in particular, so that fully opaque pixels remain fully opaque

babl: consistently use unsigned char in u8 palette conversions

... to avoid erroneous sign extensions

sse4-int8: avoid using uninitialized variable

babl: use -no-install to avoid libtool .sh wrappers

babl: fix crash when doing full introspection

also add skeleton functions for introspection of trcs/spaces

babl: do not try to enumerate sampling conversions

babl: set babl_rel_avg_error to a high value on NaN

In relation to bug #787441, if the sum of errors contain a nan the sum
is nan, we return pi as a much higher than expected average error to
indicate that this is not 0.0.

tests/grayscale_to_rgb: increase printed debug precision

tests: move non-test binaries from here to ../tools

babl: check both source/dest bits/component in bad_idea()

babl: add decrease of bits per components to bad_idea()

related to bug #787441

tests: use a epsilon instead of == for float compare

build: link extensions against internal .la again

babl: limit exported symbols

For now, also including symbols needed for internal tools and tests, that are
not part of the public API; the API promised by babl remains the public
headers, but this reduces the public dynamic loaders namespace a little.

babl: tidying of public headers

babl: in babl_polynomial_approximate_gamma(), verify that x0 >= 0

Now that the previous commit (hopefully) makes us iPhone-proof, we
can add an assertion for this condition.  If we want to replace the
assertions with some sort of bailing, we'll need to replace the rest
of them too anyway.

Bug 788093 - Crash loading (jpg with) ICC v4 profile with parametric TRCs

Fix parameter offset calculation of parametric TRCs, so that we
parse them correctry.

babl: avoid negative degree when shrinking a zero polynomial

When shrinking a zero polynomial, avoid trying to set its degree
to a negative value, and just keep it at 0, which is the minimal
degree we support.

Note that this also happens for polynomials whose coefficients are
all NaN, as is the case in bug 788093, so they also become zero
polynomials after shrinking, incidentally.

babl: make babl_space_to_icc internal, for now

Also adds an optimized 26 entry LUT, as used in the creation of
http://pippin.gimp.org/sRGBz/
 Please enter the commit message for your changes. Lines starting

babl: remove commented out debug printfs

babl: improve icc number serialization

babl: add matrix re-equalization

This optimizes the coefficients of the matrix ensuring that a RGB 1.0 1.0 1.0
results in exactly CIE Lab 100.0 0.0 0.0 and that equal R,G,B triples yield 0.0
for CIE a, b. This is achieved by rounding to 16.16 fixed point precision,
which can be exactly represented by IEEE double, and then brute-force jittering
the coefficients +/- 1 for the best solution. This is also the rounding needed
for making the matrix well behaved when used in an ICC profile.

babl: tidy up icc writing

babl: make powf approximation yield 0 for negatives

babl: improve log output on Android.

No need to log end of lines. This is not like a printf where we are
trying to pretty-print.

babl: replace a fprintf (stderr, ...) with babl_log().

It will make it visible in the logcat, for instance in Android Studio.
As a general rule, we should use babl_log() for anything which we
consider as "logs", unless we want to direct them to a given output for
sure (for instance in the case of CLI tools where we want the output to
be on standard I/O in terminal, fprintf() still is the best choice).

tests: make babl build on haiku

Using -lphtread instead of -pthread in Makefile.am fixes the build when using
gcc-x86 under haiku, figured out by schumaml.

babl/base: model-gray, avoid potential NaN

babl: add rendering intent argument to icc loading API

babl: abstract out TRC gamma detectors

babl: babl_log() implementation for Android.

babl: skip unknown formats when loading cache

Fixes bug #787467.

Formats based on custom primaries provided by applications are unknown to babl
at launch, we still want to keep the data though, since this list of most used
fishes first is useful resource in combination with the warnings about missing
fast paths.

babl: make utility file reading function non-static

babl: remove inlined implementation of powf

babl: replace trc lut inverter with binary search

Much faster and much more accurate.

babl: add icc meta-data extraction functions

Instead of extracting all meta-data of ICC profiles and attaching them to
BablSpaces, we provide an string key/value API to query meta data of in-memory
ICC profiles.

tools: add BablPolynomial test to trc-validator.c

babl: in BablTRC, use BablPolynomial to approximate gamma func.

Use BablPolynomial to approximate gamma functions of TRCs over the
(0, 1) range, for FORMULA_GAMMA and FORMULA_SRGB TRCs.

Use generic FORMULA_GAMMA TRCs for 1.8 and 2.2 gammas, instead of
specialized versions.

babl: add babl-polynomial.[hc]

Implements BablPolynomial, an opaque type representing a real valued
polynomial of a real variable.  Currently, exposes the following
functions:

  * babl_polynomial_eval():  Evaluates a polynomial.

  * babl_polynomial_approximate_gamma():  Calculates a polynomial
    approximation of a gamma function.

babl: further nan prevention in babl-trc

babl: in babl-trc.c, fix babl_powf()

babl: avoid accidental matrix overwrite by long names

.. by increasing the static per space allocation to 512 bytes, these names need
to become shorter - or the name be allocated dynamically. This fixes problems
with parameterized lab / sRGB like TRCs - which yielded really long names.

babl: protect some functions used by TRC code from NaNs

babl: actually reverse preceding change

restore segfault for massimo

@massimo: pretty please use git :)

It would be really nice if others, and prolifix people like
you, prepared git formatted patches, so that it is much saner
and easier to push and work with contributions as a maintainer
with a dirty tree, and in even more valuable bugs than this
one.. without needing to re-read and re-understand the entire
context of the fix, before fully reimplementing it.

This commit fixes bug #787441, where NaN is polluting the
computer error/tolerance.

extensions: finish API update

NEWS: aggregate news

update docs

extensions: follow change dropping long return

babl: drop long return from conversion funcs

They are propagated and instructions are spent on that, but their value are
never used. Some conversion functions in base even used the argument passed for
counting down to 0 and return 0 instead, to no ill effect.

Since we do not currently see external babl extensions, we can break extensions
API without bumping the library version, this is even a change where due to
varargs, old extensions will continue working until migrated.

babl: remove unused static linear luts for u8 rgb space converters

extensions: keep old algoritmic float in parallel with table

The newly introduced table isn't bit accurate with our reference,
the ~3x faster float 2 half conversion is available with a higher
tolerance.

extensions: fix type error in float 2 half

extensions: make float-half extension use exact LUTs

Both for conversion from half to float with a on load initialized table, and
with new tables an faster approach from qcms / mozilla / webkit.

babl: make more use of float matrices where appropriate

babl: trade gained fishing performance for better fish

babl: bail early on paths that are bad ideas

Not going through/bench-marking chains that will make the number of components
drop below the input/output counts, and not permitting a temporary lack of
alpha both fixes bugs at higher tolerances and speeds up path selection.

babl: add babl_fast_fish api

babl: add R'G'B' u8 color space convertors

babl: make TRCs internal API

babl: add Rec2020 space

babl: make matrix+trc space detector generic

babl: abstract space duplicate detection

tools: follow constness change in icc parsing error handling

babl: make error argument of babl_space_from_icc optional

babl: detect sRGB profiles from matrix

When TRCs also are matching, return internal sRGB instead of creating a
duplicate.

babl: indentation in trc

babl: move srgb detection to srgb_formula constructor

babl: fix overflow detection at end of luts

babl: implement interpolation for lut trcs

babl: icc implement ascii decoding of mluc

babl: add parametric handling for sRGB style TRCs (icc V4)

babl: use trcs own fun directly in early stages

tools: babl-icc-rewrite; abort with error message instead of segfault

babl: use babl_malloc instead of misgided use of aligned_alloc thinking it was like alloca

babl: use _aligned_malloc() instead of aligned_alloc() on Win32.

Thanks to Lionel N. for raising the issue. It was failing to build for
Windows with:
> babl/babl-space.c:503: undefined reference to `aligned_alloc'
Windows has apparently a similar API, except that the size and alignment
parameters are inverted.
See https://msdn.microsoft.com/library/8z34s9c6.aspx

babl-icc: add support for some ICCv4 profiles

RGB Matrix profiles with not too complex (or sRGB) parametric TRCs is what
should work fine.